Skip to content

docs: improve observability / monitoring documentation#310

Open
Georg (D3luxee) wants to merge 15 commits into
mainfrom
gd/observability_docs
Open

docs: improve observability / monitoring documentation#310
Georg (D3luxee) wants to merge 15 commits into
mainfrom
gd/observability_docs

Conversation

@D3luxee

Copy link
Copy Markdown
Contributor

Goal of this PR

Improve monitoring / observability documentation

Copilot AI review requested due to automatic review settings June 24, 2026 09:14
@D3luxee Georg (D3luxee) requested a review from a team as a code owner June 24, 2026 09:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds documentation for the Multiplayer Servers “observability tap” feature and links it into the Monitoring sidebar, so readers can discover how to access Prometheus-compatible metrics and Loki-compatible logs endpoints.

Changes:

  • Adds a new Monitoring page describing observability tap access, endpoints, and Grafana data source setup.
  • Adds a new sidebar navigation entry under Monitoring for the new page.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/multiplayer-servers/monitoring/sidebar.json Adds a new sidebar item linking to the observability tap page.
src/multiplayer-servers/monitoring/obs-tap.md Introduces the observability tap documentation, including endpoints and Grafana setup steps.

Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md
Comment thread src/multiplayer-servers/monitoring/obs-tap.md
Comment thread src/multiplayer-servers/monitoring/sidebar.json Outdated

@antiphp Christian R (antiphp) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What AI says and:

  • For me "tap" is not a familiar term. If it is, ignore me.
  • Wonder if you want to mention what metrics the user can expect, GameFabric metrics or Agones Cluster metrics, or both. Maybe add one example each, so the user has a starting point?
  • Same for the Loki endpoints. Just a sentence that it allows accessing the game server logs, see further details here/there (if we have something to link).

Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Comment thread src/multiplayer-servers/monitoring/introduction.md Outdated
Comment thread src/multiplayer-servers/monitoring/introduction.md Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
### Metrics data source

1. In Grafana, go to **Connections > Data sources** and select **Add new data source**.
1. Choose **Prometheus** as the data source type.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's up with the 1. 1. 1. 1. numeration here? Maybe the markdown renderer will auto-fix this, but I'm not sure

Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md Outdated
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Copilot AI review requested due to automatic review settings June 24, 2026 13:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

Comment thread src/multiplayer-servers/monitoring/sidebar.json Outdated
Comment thread src/multiplayer-servers/monitoring/obs-tap.md

The metrics endpoint is compatible with the [VictoriaMetrics MetricsQL](https://docs.victoriametrics.com/metricsql/) query language, which is a superset of PromQL. Any tool that supports PromQL also works against this endpoint.

The following Prometheus HTTP API paths are available under the metrics endpoint:
Comment thread src/multiplayer-servers/monitoring/obs-tap.md
Copilot AI review requested due to automatic review settings June 24, 2026 13:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Comment thread src/multiplayer-servers/monitoring/obs-tap.md
Comment thread src/multiplayer-servers/monitoring/introduction.md Outdated
Comment thread src/multiplayer-servers/monitoring/introduction.md
Copilot AI review requested due to automatic review settings June 24, 2026 15:01
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
@D3luxee Georg (D3luxee) changed the title docs: document observability tap docs: improve observability / monitoring documentation Jun 24, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comment on lines +65 to +70
1. In Grafana, go to **Connections > Data sources** and select **Add new data source**.
1. Choose **Prometheus** as the data source type.
1. Set the **URL** to `https://<your-installation>.gamefabric.dev/observability/metrics/`.
4. Under **Authentication**, set the **Authentication method** to **No authentication**
5. Under **HTTP headers**, click **Add header** and set **Header** to `Authorization` and **Value** to `Bearer <token>`. Replace `<token>` with the token you receive from your Customer Success Manager. Keep the `Bearer` prefix.
1. Select **Save & test** to verify the connection.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In other parts of our docs, like for example https://github.com/GameFabric/docs/blob/main/src/multiplayer-servers/getting-started/quickstart.md?plain=1#L83-L87, we use manual numbering. Auto-numbering might also work, but for consistency and to be on the safe side, I would lean more towards manual numbering.

Comment on lines +74 to +79
1. In Grafana, go to **Connections > Data sources** and select **Add new data source**.
1. Choose **Loki** as the data source type.
1. Set the **URL** to `https://<your-installation>.gamefabric.dev/observability/logs/`.
4. Under **Authentication**, set the **Authentication method** to **No authentication**
5. Under **HTTP headers**, click **Add header** and set **Header** to `Authorization` and **Value** to `Bearer <token>`. Replace `<token>` with the token you receive from your Customer Success Manager. Keep the `Bearer` prefix.
1. Select **Save & test** to verify the connection.
Copilot AI review requested due to automatic review settings June 24, 2026 16:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

3. Set the **URL** to `https://<your-installation>.gamefabric.dev/observability/metrics/`.
4. Under **Authentication**, set the **Authentication method** to **No authentication**
5. Under **HTTP headers**, click **Add header** and set **Header** to `Authorization` and **Value** to `Bearer <token>`. Replace `<token>` with the token you receive from your Customer Success Manager. Keep the `Bearer` prefix.
1. Select **Save & test** to verify the connection.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants